-
Notifications
You must be signed in to change notification settings - Fork 1
경험/문항/채팅 API 데이터 레이어 구현 #16
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: develop
Are you sure you want to change the base?
Conversation
- Chatting Source
- Json provide 메서드 - ConverterFactory 부착 - CallAdapterFactory 부착
- 매핑 로직 - Experience 도메인 모델 - Hilt 모듈
| internal interface RepositoryModule { | ||
|
|
||
| @Binds | ||
| @ActivityScoped |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ActivityRetainedComponent는 화면 회전 시에도 유지되게 하는 설정값이고 ActivityScoped는 무조건 Activity 생명주기에 따라 소멸하는 설정값인 것 같은데(잘 모름)
이거 혹시 @ActivityRetainedScoped를 써야 의도대로 동작하지 않을까요?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
앗 이런!! 수정할게여
|
|
||
| fun startChattingStream(questionId: String, sendingMessage: String): Flow<ChattingStreaming> | ||
| suspend fun getChatHistory(questionId: String): Result<ChattingHistory> | ||
| suspend fun updateLetter(chattingId: String, questionId: String, content: String): Result<Unit> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
혹시 이게 자소서 업데이트하는 함수라면 updatePaper로 이름 지으면 어떨까요?
왜냐하면 Figma에서 자소서 아이콘 이름이 paper라서.. ㅎㅎ
아니면 아예 cover letter라고 표현하는 것도 괜찮을 것 같습니다
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
이게 백엔드에서는 Draft로 표현하는거 같더라구요 ...
일단 다같이 얘기를 해봐야할듯 합니다 !!
이슈
내용
위 API들에 대한 데이터 레이어 구조를 구현했습니다 (Repository - DataSource - Retrofit)
[또 1000줄을 넘겨버렸는데... 대부분 단순 중복이라 훅훅 넘겨주세요 ㅠ]